Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🧑🏼‍🏫 Add support for sphinx exercise #399

Merged
merged 5 commits into from
May 28, 2023
Merged

Conversation

rowanc1
Copy link
Collaborator

@rowanc1 rowanc1 commented May 28, 2023

This PR adds support for sphinx-exercise, cc @mmcky as well as a number of small improvements around cross-references (no nested links inside a cross-ref link) as well as general support for gated directives.

exercise

See jupyter-book/myst-theme#104

Copy link
Collaborator Author

@rowanc1 rowanc1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few notes!

@@ -49,6 +57,20 @@ function getDefaultNumberedReferenceLabel(kind: TargetKind | string) {
}
}

function getDefaultNamedReferenceLabel(kind: TargetKind | string, hasTitle: boolean) {
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This function should eventually be moved to be added/accessible from the directives. Currently this is hard-coded in the transforms.

@rowanc1 rowanc1 merged commit 841d5b8 into main May 28, 2023
@rowanc1 rowanc1 deleted the feat/exercise branch May 28, 2023 22:19
@rowanc1
Copy link
Collaborator Author

rowanc1 commented May 28, 2023

@mmcky @choldgraf the docs for the sphinx exercise extension are here:
https://myst-tools.org/docs/mystjs/exercises

Let me know your thoughts!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant